Skip to content

Update MANIFEST.in so tests are in the sdist#11

Merged
pombredanne merged 1 commit intoaboutcode-org:developfrom
kitterma:patch-1
Jan 26, 2020
Merged

Update MANIFEST.in so tests are in the sdist#11
pombredanne merged 1 commit intoaboutcode-org:developfrom
kitterma:patch-1

Conversation

@kitterma
Copy link
Copy Markdown
Contributor

I've just packaged publicsuffix2 for Debian (so we can get rid of publicsuffix). We like to run the package tests are part of our QA process, so it would be nice if they were included in the sdist.

I've just packaged publicsuffix2 for Debian (so we can get rid of publicsuffix).  We like to run the package tests are part of our QA process, so it would be nice if they were included in the sdist.
@pombredanne
Copy link
Copy Markdown
Member

@kitterma thank you ++ for the Debianization! feel free to upstream it if you like.
Let me merge this alright. I usually avoid to package tests in the sdists, but that's minor I reckon his makes sense when packaging alright!

@pombredanne pombredanne merged commit 1332a4b into aboutcode-org:develop Jan 26, 2020
@kitterma
Copy link
Copy Markdown
Contributor Author

Thank you. Most of the debianization is to us the system PSL, so I don't think there's a lot that's relevant to upstream. I pulled the tests in as a patch, but I'll drop that after the next release (which I appreciate because there's always a risk of getting out of sync the way I'm doing it at the moment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants